Skip to content

Conversation

@aka-sacci-ccr
Copy link
Contributor

@aka-sacci-ccr aka-sacci-ccr commented Oct 14, 2025

What is this Contribution About?

Please provide a brief description of the changes or enhancements you are proposing in this pull request.

Issue Link

Please link to the relevant issue that this pull request addresses:

Loom Video

Record a quick screencast describing your changes to help the team understand and review your contribution. This will greatly assist in the review process.

Demonstration Link

Provide a link to a branch or environment where this pull request can be tested and seen in action.

Summary by CodeRabbit

  • New Features

    • Archive a single HubSpot conversation thread.
    • Update a conversation thread’s status (OPEN/CLOSED) and archived state.
  • Refactor

    • Updated the available HubSpot conversation actions to include archiving and updating threads.
    • Removed legacy actions for sending thread messages and comments from the catalog.

@aka-sacci-ccr aka-sacci-ccr changed the title added archive and update thread Hubspot: added archive and update thread Oct 14, 2025
@github-actions
Copy link
Contributor

Tagging Options

Should a new tag be published when this PR is merged?

  • 👍 for Patch 0.128.10 update
  • 🎉 for Minor 0.129.0 update
  • 🚀 for Major 1.0.0 update

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 14, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Adds two new HubSpot conversation thread actions: one to archive a thread via DELETE and one to update a thread via PATCH. Updates the generated manifest to register these actions and removes prior sendThreadComment/sendThreadMessage entries.

Changes

Cohort / File(s) Change Summary
New conversation thread actions
hubspot/actions/conversations/archiveThread.ts, hubspot/actions/conversations/updateThread.ts
Added archiveThread (DELETE /conversations/v3/conversations/threads/{threadId}) returning void; added updateThread (PATCH …/{threadId}) with Props, ThreadAssociations, ThreadResponse typings, returning API response.
Manifest updates
hubspot/manifest.gen.ts
Registered new actions (archiveThread, updateThread). Removed sendThreadComment and sendThreadMessage from action mapping.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Client
  participant Action as updateThread action
  participant HSClient as HubSpotClient
  participant HSAPI as HubSpot Conversations API

  Client->>Action: call updateThread({ threadId, archived?, status? })
  Action->>HSClient: instantiate with ctx
  Action->>HSAPI: PATCH /conversations/v3/conversations/threads/{threadId}\n body: { archived?, status? }
  HSAPI-->>Action: 200 ThreadResponse
  Action-->>Client: ThreadResponse
  note over Action,HSAPI: No additional branching or error handling
Loading
sequenceDiagram
  autonumber
  actor Client
  participant Action as archiveThread action
  participant HSClient as HubSpotClient
  participant HSAPI as HubSpot Conversations API

  Client->>Action: call archiveThread({ threadId })
  Action->>HSClient: instantiate with ctx
  Action->>HSAPI: DELETE /conversations/v3/conversations/threads/{threadId}
  HSAPI-->>Action: 204 No Content
  Action-->>Client: void
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

I hop through threads, tidy and neat,
Patch a status, make deletions discreet.
With a flick of ears, I file and prune,
Inbox moons hum a softer tune.
Tap-tap paws—archive, update—complete! 🐇✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/hubspot-thread-updates

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b7628b8 and c246ee9.

📒 Files selected for processing (3)
  • hubspot/actions/conversations/archiveThread.ts (1 hunks)
  • hubspot/actions/conversations/updateThread.ts (1 hunks)
  • hubspot/manifest.gen.ts (2 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@aka-sacci-ccr aka-sacci-ccr merged commit f5b75ee into main Oct 14, 2025
2 of 4 checks passed
@aka-sacci-ccr aka-sacci-ccr deleted the feat/hubspot-thread-updates branch October 14, 2025 17:13
@coderabbitai coderabbitai bot mentioned this pull request Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants